-
Notifications
You must be signed in to change notification settings - Fork 1
[BUILD FAIL] Feature/137 Feature/137 봉사활동 모집글, 커뮤니티 게시글 검색 조회 (elastic search) #215
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
- keyword로 검색 결과 반환 - elasticsearch에 저장 - 전체 데이터 조회 - elasticsearch 데이터 삭제
- elasticsearch 저장 - elasticsearch 키워드 검색 - 전체글 조회
- application-test.yml 수정 - elastic search uris -> uri test(search): 코드 리뷰 사항 반영 - application-test.yml 수정
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
빌드에 실패했습니다.
m-a-king
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨습니다! 쿼리 고수가 되셨네요.
테스트 해결해야 할 텐데 왜 안되는 걸까요?
그리고 테스트에서 뭔가 매직 넘버들이 많은데 따라가는 게 조금 헷갈리는 것 같았어요.
| private List<CommunityBoardDocument> convertEntityToDocuments(List<CommunityBoard> communityBoards) { | ||
| List<CommunityBoardDocument> communityBoardDocuments = new ArrayList<>(); | ||
|
|
||
| for (CommunityBoard communityboard : communityBoards) { | ||
| CommunityBoardDocument document = CommunityBoardDocument.builder() | ||
| .id(communityboard.getId()) | ||
| .title(communityboard.getTitle()) | ||
| .content(communityboard.getContent()) | ||
| .build(); | ||
| communityBoardDocuments.add(document); | ||
| } | ||
| return communityBoardDocuments; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
서비스에 있으면 좋을 것 같은데 혹시 여기 있는 이유가 있나요?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
documentRepository의 saveAll 사용하려고 repository에서 document로 바꿔주었습니다!
| this.title = title; | ||
| this.content = content; | ||
| } | ||
| } No newline at end of file |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
끄덕
📌 과제 설명
👩💻 요구 사항과 구현 내용
✅ PR 포인트 & 궁금한 점